Skip to content

feat: add riscv64 support to yq toolchain #1066

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

kxxt
Copy link

@kxxt kxxt commented Apr 12, 2025

This patch adds riscv64 support to yq toolchain.

The last version 4.33.3 in lib/private/yq_toolchain.bzl does not have riscv64 binary releases in yq repository so I added the latest release to it.

Copy link
Collaborator

@alexeagle alexeagle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@alexeagle alexeagle enabled auto-merge (squash) April 23, 2025 00:12
kxxt added 2 commits April 23, 2025 02:40
yq's automatic output format changed and it starts to output JSON for
JSON inputs.
Add `-oy` to explicitly specify that we want yaml.
yq starts to add some xml metadata to the yaml output.
auto-merge was automatically disabled April 23, 2025 00:45

Head branch was pushed to by a user without write access

@kxxt
Copy link
Author

kxxt commented Apr 23, 2025

It looks that yq update breaks some tests. I have pushed some fixes.
PTAL again. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants